src/pages/food-truck-detail/hooks/use-food-truck-detail.ts (1)

13-13: ID 검증 로직을 더 명확하게 개선하세요.

enabled: !!foodTruckId는 0을 허용합니다. 푸드트럭 ID가 1부터 시작한다면 enabled: foodTruckId > 0이 더 명확합니다.

🔎 제안하는 수정

-  enabled: !!foodTruckId,
+  enabled: foodTruckId > 0,

src/pages/@owner/food-truck-form/hooks/use-food-truck-form.ts (1)

56-80: 여러 번의 리렌더링 가능성을 고려해주세요.

foodTruckDetailData와 menuData가 서로 다른 시점에 로드되면 폼이 여러 번 리셋될 수 있습니다. 또한 API 호출 실패 시 에러 처리가 없습니다.

💡 한 번만 리셋하도록 개선하는 제안

두 데이터가 모두 준비된 후 한 번만 리셋하도록 수정할 수 있습니다:

  useEffect(() => {
-    if (foodTruckDetailData) {
+    if (foodTruckDetailData && menuData !== undefined) {
      setIsEdit(true);
      reset({
        // ... 기존 코드
      });
    }
  }, [foodTruckDetailData, menuData, reset]);

또는 두 API 호출이 모두 완료될 때까지 대기하는 로딩 상태를 추가할 수 있습니다.

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

src/pages/@owner/food-truck-form/FoodTruckForm.tsx (1)

69-72: return 문이 누락되어 잘못된 렌더링이 발생합니다.

유효성 검증 후 return null이 없어서 토스트와 navigate 호출 후에도 컴포넌트가 계속 렌더링됩니다. 이는 예상치 못한 동작과 에러를 발생시킬 수 있습니다.

🔎 제안하는 수정

  if (!foodTruckId || isNaN(foodTruckIdNumber)) {
    toast.error('잘못된 접근입니다.');
    navigate(ROUTES.FOOD_TRUCK_MANAGEMENT);
+   return null;
  }

더 나은 사용자 경험을 위해 useEffect로 처리하는 방법도 고려하세요:

+ useEffect(() => {
+   if (!foodTruckId || isNaN(foodTruckIdNumber)) {
+     toast.error('잘못된 접근입니다.');
+     navigate(ROUTES.FOOD_TRUCK_MANAGEMENT);
+   }
+ }, [foodTruckId, foodTruckIdNumber, toast, navigate]);
+
  if (!foodTruckId || isNaN(foodTruckIdNumber)) {
-   toast.error('잘못된 접근입니다.');
-   navigate(ROUTES.FOOD_TRUCK_MANAGEMENT);
+   return null;
  }

src/shared/utils/normalize-enum-value.ts (1)

1-11: LGTM!

유틸리티 함수의 로직이 올바르며 enum 값 정규화를 안전하게 처리합니다.

💡 더 간결한 구현 제안 (선택사항)

타입 캐스팅을 줄이고 가독성을 개선할 수 있습니다:

 export const normalizeEnumValue = <T extends Record<string, string>>(
   enumObj: T,
   value?: string
 ): T[keyof T] | undefined => {
   if (!value) return undefined;
 
   const values = Object.values(enumObj) as Array<T[keyof T]>;
-  return values.includes(value as T[keyof T])
-    ? (value as T[keyof T])
-    : undefined;
+  return values.includes(value as T[keyof T]) ? value as T[keyof T] : undefined;
 };

src/pages/@owner/food-truck-form/hooks/use-food-truck-form.ts (1)

61-95: useEffect 의존성 배열을 최적화하세요.

menuData가 의존성에 포함되어 메뉴 페이지가 로드될 때마다 useEffect가 재실행됩니다. infinite query의 특성상 스크롤 시에도 트리거될 수 있습니다.

🔎 제안하는 최적화

데이터 로딩 완료 시점만 감지하도록 변경:

+ const isDataLoaded = !!foodTruckDetailData && menuData !== undefined;
+
  useEffect(() => {
    if (foodTruckDetailData) {
      const menus = menuData?.pages.flatMap(page => page?.content || []);
      setIsEdit(true);
      reset({
        // ... form data
      });
    }
- }, [foodTruckDetailData, menuData, reset]);
+ }, [isDataLoaded, reset]);

또는 초기 로드만 처리:

+ const [isInitialized, setIsInitialized] = useState(false);
+
  useEffect(() => {
-   if (foodTruckDetailData) {
+   if (foodTruckDetailData && !isInitialized) {
      const menus = menuData?.pages.flatMap(page => page?.content || []);
      setIsEdit(true);
      reset({
        // ... form data
      });
+     setIsInitialized(true);
    }
- }, [foodTruckDetailData, menuData, reset]);
+ }, [foodTruckDetailData, menuData, isInitialized, reset]);

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

src/pages/@owner/food-truck-form/hooks/use-food-truck-form.ts (2)

78-81: enum 정규화 실패 시 UX를 개선하는 것을 고려하세요.

normalizeEnumValue가 undefined를 반환할 때 에러 토스트를 표시하고 early return하는 것은 좋은 방어 로직입니다. 다만 사용자는 빈 폼만 보게 되므로, 필요시 기본값으로 폼을 채우거나 재시도 옵션을 제공하는 것을 고려해보세요.

Based on past review comments, 이 로직은 이전 리뷰 피드백을 반영하여 normalizeEnumValue의 undefined 반환을 적절히 처리하고 있습니다.

---

101-101: 메뉴 체크 로직을 단순화할 수 있습니다.

flatMap은 항상 배열을 반환하므로 menus !== undefined 체크는 불필요합니다. menus.length > 0만으로 충분합니다.

🔎 단순화 제안

-        menus: menus !== undefined && menus.length !== 0,
+        menus: menus.length > 0,

Based on past review comments, 이 로직은 이전 리뷰 피드백을 반영하여 menuData.pages를 올바르게 검사하고 실제 메뉴 컨텐츠가 있는지 확인합니다.

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

src/pages/@owner/food-truck-form/@section/region-section/RegionSection.tsx (1)

35-44: 불필요한 null 체크를 간소화할 수 있습니다.

Line 35에서 formData.regionCodes ?? []로 이미 빈 배열을 기본값으로 설정했으므로, Line 41의 regionCodes && 체크는 불필요합니다.

🔎 수정 제안

       <RegionButton
         text={
-          regionCodes && regionCodes.length > 0
+          regionCodes.length > 0
             ? regionCodes.map(region => region.name).join(', ')
             : ''
         }

src/pages/@owner/set-region/hooks/use-region.ts (1)

15-47: 중복된 가드 로직을 추출하여 DRY 원칙을 적용할 수 있습니다.

handleSubmitRegion과 handleResetRegionFoodTruck 모두 동일한 foodTruckId 검증 로직을 포함하고 있습니다. 헬퍼 함수로 추출하면 유지보수성이 향상됩니다.

🔎 수정 제안

+  const navigateToForm = (regionCodes: RegionResponse[]) => {
+    if (!foodTruckId) {
+      toast.error('잘못된 접근입니다.');
+      navigate(ROUTES.FOOD_TRUCK_MANAGEMENT);
+      return;
+    }
+    navigate(ROUTES.FOOD_TRUCK_FORM(foodTruckId), {
+      state: {
+        from: 'set-region',
+        formData: {
+          ...formData,
+          regionCodes,
+        },
+      },
+    });
+  };
+
   const handleSubmitRegion = (regions: RegionResponse[]) => {
-    if (!foodTruckId) {
-      toast.error('잘못된 접근입니다.');
-      navigate(ROUTES.FOOD_TRUCK_MANAGEMENT);
-      return;
-    }
-    navigate(ROUTES.FOOD_TRUCK_FORM(foodTruckId), {
-      state: {
-        from: 'set-region',
-        formData: {
-          ...formData,
-          regionCodes: regions,
-        },
-      },
-    });
+    navigateToForm(regions);
   };

   const handleResetRegionFoodTruck = () => {
-    if (!foodTruckId) {
-      toast.error('잘못된 접근입니다.');
-      navigate(ROUTES.FOOD_TRUCK_MANAGEMENT);
-      return;
-    }
-    navigate(ROUTES.FOOD_TRUCK_FORM(foodTruckId), {
-      state: {
-        from: 'set-region',
-        formData: {
-          ...formData,
-          regionCodes: [],
-        },
-      },
-    });
+    navigateToForm([]);
   };

src/pages/@owner/food-truck-form/utils/reset-food-truck-form-value.ts (1)

12-56: 반환 타입을 명시적으로 정의하면 타입 안전성이 향상됩니다.

현재 암시적 반환 타입 대신 discriminated union을 사용하면 호출 측에서 타입 가드를 더 명확하게 사용할 수 있습니다.

🔎 수정 제안

type ResetFormResult =
  | { isError: true; values?: never }
  | { isError: false; values: FoodTruckFormData };

export const resetFoodTruckFormValue = (
  foodTruckDetailData: FoodTruckDetailResponse,
  menus?: MyFoodTruckMenuResponse[]
): ResetFormResult => {
  // ... existing implementation
};

src/pages/@owner/food-truck-form/hooks/use-food-truck-form.ts (1)

71-82: TODO 코멘트를 이슈로 관리하시겠습니까?

Line 79에 푸드트럭 등록 API 호출이 TODO로 남아있습니다. PR 설명에서 언급하신 것처럼 아직 API 연동 전이라는 점을 이해합니다.

이 TODO를 별도 이슈로 생성하여 추적하시겠습니까? 생성을 원하시면 알려주세요.

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

src/pages/@owner/set-region/SetRegion.tsx (2)

36-36: location.state null 안전성이 제거되어 런타임 에러가 발생할 수 있습니다.

이전 리뷰에서 지적되었던 것처럼, 사용자가 URL로 직접 접근하거나 브라우저 히스토리를 통해 접근할 경우 location.state가 null일 수 있습니다.

현재 코드는 옵셔널 체이닝 없이 location.state.formData에 접근하므로, state가 null일 때 런타임 에러가 발생합니다. 57-60번 줄의 formData 체크는 이 접근 이후에 실행되므로 에러를 방지할 수 없습니다.

🔎 수정 제안

 function SetRegionContent() {
   const { foodTruckId } = useParams();
   const navigate = useNavigate();
   const location = useLocation();
-  const formData = location.state.formData;
+  const formData = location.state?.formData;
+
+  if (!formData) {
+    navigate(ROUTES.FOOD_TRUCK_MANAGEMENT);
+    return null;
+  }

   const handleLeftClick = () => {
     if (!foodTruckId) {
       navigate(ROUTES.FOOD_TRUCK_MANAGEMENT);
       return;
     }
     const fromPage = location.state?.from;
     navigate(ROUTES.FOOD_TRUCK_FORM(foodTruckId), {
       state: {
         from: fromPage || 'food-truck-form',
         formData: formData,
       },
     });
   };

   const { handleSubmitRegion, handleResetRegionFoodTruck } = useRegion(
     formData,
     foodTruckId
   );

-  if (!formData) {
-    navigate(ROUTES.FOOD_TRUCK_MANAGEMENT);
-    return null;
-  }

   return (

---

16-23: 유효성 검사가 훅 호출 이후에 실행되는 치명적인 순서 문제가 있습니다.

18번 줄에서 useFoodTruckForm이 호출된 이후에 20-23번 줄에서 유효성 검사가 수행됩니다. 이로 인해:

1. 유효하지 않은 foodTruckId(NaN)가 이미 훅에 전달됨
2. navigate() 호출 후에도 컴포넌트가 계속 렌더링되어 예상치 못한 동작 발생
3. 유효성 검사가 실질적으로 아무것도 방지하지 못함

React의 훅 규칙상 조건부 렌더링이 필요한 경우 유효성 검사를 훅 호출 이전에 수행하고 early return 해야 합니다.

🔎 수정 제안

 export default function SetRegion() {
   const navigate = useNavigate();
   const toast = useToast();

   const { foodTruckId } = useParams();
   const foodTruckIdNumber = Number(foodTruckId);
-  const methods = useFoodTruckForm(foodTruckIdNumber);

   if (!foodTruckId || isNaN(foodTruckIdNumber)) {
     toast.error('잘못된 접근입니다.');
     navigate(ROUTES.FOOD_TRUCK_MANAGEMENT);
+    return null;
   }
+
+  const methods = useFoodTruckForm(foodTruckIdNumber);

   return (
     <FormProvider {...methods.methods}>
       <SetRegionContent />
     </FormProvider>
   );
 }

src/shared/components/region/hooks/use-region.ts (1)

21-32: 이전 리뷰 피드백이 반영되어 개선되었습니다.

토글 로직이 명확하게 구현되었고, toast.error() 호출이 setState 업데이터 외부로 이동되어 이전 리뷰에서 지적된 사이드 이펙트 문제가 해결되었습니다.

다만, 빠른 연속 클릭 시 selectedRegions.length 체크(27번 줄)가 stale closure 값을 읽을 가능성이 있습니다. 더 엄격한 동시성 보장이 필요하다면 모든 상태 체크를 단일 함수형 setState 내부로 이동하는 것을 고려할 수 있습니다.

🔎 선택적 개선 사항

 const handleSelectRegion = (region: RegionResponse) => {
-  const alreadySelected = selectedRegions.some(r => r.code === region.code);
-  if (alreadySelected) {
-    setSelectedRegions(prev => prev.filter(r => r.code !== region.code));
-    return;
-  }
-  if (selectedRegions.length >= MAX_SELECTED) {
-    toast.error('최대 선택 개수를 초과했습니다.');
-    return;
-  }
-  setSelectedRegions(prev => [...prev, region]);
+  setSelectedRegions(prev => {
+    const alreadySelected = prev.some(r => r.code === region.code);
+    
+    if (alreadySelected) {
+      return prev.filter(r => r.code !== region.code);
+    }
+    
+    if (prev.length >= MAX_SELECTED) {
+      return prev;
+    }
+    
+    return [...prev, region];
+  });
+  
+  // 토스트는 상태 업데이트 후 useEffect나 별도 체크로 처리
+  if (!selectedRegions.some(r => r.code === region.code) && 
+      selectedRegions.length >= MAX_SELECTED) {
+    toast.error('최대 선택 개수를 초과했습니다.');
+  }
 };

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

src/pages/@owner/upload-food-truck-images/hooks/use-food-truck-image.ts (1)

42-55: 사용자 피드백 개선 및 일관성 검토 권장

토스트 알림 추가로 사용자 피드백이 개선되었습니다. 다만 다음 사항들을 고려해보시기 바랍니다:

1. 에러 메시지 구체성: 현재 onError는 범용 메시지('이미지 업로드 오류')를 표시하지만, API에서 던지는 구체적인 에러 정보를 활용하면 사용자가 문제를 더 쉽게 파악할 수 있습니다.

2. 일관성: 같은 파일의 useDeleteImage 훅도 사용자 작업이지만 토스트 피드백이 없어 UX 일관성이 떨어집니다. 이미지 삭제 작업에도 성공/실패 알림 추가를 검토해보세요.

🔎 에러 메시지 구체화 예시

    onError: () => {
-     toast.error('이미지 업로드 오류');
+     toast.error(error.message || '이미지 업로드 오류');
    },

또는 에러 파라미터를 활용:

-   onError: () => {
+   onError: (error) => {
      toast.error('이미지 업로드 오류');
    },

src/pages/@owner/upload-food-truck-images/hooks/use-upload-images.ts (1)

19-40: formData에 타입 정의를 추가하는 것을 고려하세요.

formData가 타입 어노테이션 없이 사용되고 있어 photoUrls 같은 속성의 존재가 컴파일 타임에 보장되지 않습니다. 명시적인 타입을 정의하면 타입 안전성과 IDE 자동완성이 개선됩니다.

🔎 타입 정의 예시

별도의 타입 파일에 인터페이스를 정의:

// types/food-truck-form.ts
export interface FoodTruckFormData {
  photoUrls?: string[];
  // 다른 폼 필드들...
}

그리고 훅에서 사용:

+import type { FoodTruckFormData } from '@/types/food-truck-form';
+
 export const useUploadImages = () => {
   const navigate = useNavigate();
   const location = useLocation();
   const { foodTruckId } = useParams<{ foodTruckId: string }>();
-  const formData = location.state?.formData;
+  const formData = location.state?.formData as FoodTruckFormData | undefined;

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

src/pages/@owner/food-truck-form/hooks/use-food-truck-form.ts (1)

89-89: isValid 체크가 중복됩니다.

handleSubmit으로 래핑된 콜백 함수 내부에서 isValid 체크는 불필요합니다. handleSubmit이 이미 폼 유효성 검사를 수행한 후에만 콜백을 호출하기 때문입니다.

🔎 간소화 제안

-    if (isValid && formData) {
+    if (formData) {
      // TODO: 서버에 필요한 형식에 맞게 포맷하여 보내기. 타입 만들어야 함
-      alert('푸드트럭 등록 제출');
      updateFoodTruckInfo({

또는 formData는 항상 존재하므로 조건문 자체를 제거할 수 있습니다:

-    if (isValid && formData) {
-      // ...
-    }
+    // TODO: 서버에 필요한 형식에 맞게 포맷하여 보내기. 타입 만들어야 함
+    updateFoodTruckInfo({
+      // ...
+    });

src/pages/@owner/menu/hooks/use-menu-register.ts (1)

8-10: 타입 안전성 및 변수명 개선 제안

현재 location.state?.formData에 대한 타입 정의가 없어 런타임 에러 위험이 있습니다. 또한 메뉴 등록 훅에서 foodTruckFormData라는 이름은 혼란을 줄 수 있습니다.

다음과 같이 개선하는 것을 권장합니다:

1. location.state에 대한 타입 정의 추가
2. 변수명을 맥락에 맞게 변경 (예: preservedFormData 또는 foodTruckFormState)

제안하는 개선 방안

타입 정의를 추가하고 변수명을 개선:

+import type { FoodTruckFormData } from '@pages/@owner/food-truck-form/types'; // 실제 타입 경로로 수정 필요
+
+interface LocationState {
+  formData?: FoodTruckFormData;
+}
+
 export const useRegisterMenu = (foodTruckId: number) => {
   const navigate = useNavigate();
-  const location = useLocation();
+  const location = useLocation() as { state?: LocationState };
 
-  const foodTruckFormData = location.state?.formData;
+  const preservedFormData = location.state?.formData;

src/pages/@owner/menu/hooks/use-menu-mutations.ts (1)

21-23: 코드 중복 제거 권장

동일한 패턴이 세 개의 mutation hook에서 반복됩니다. 커스텀 훅으로 추출하여 재사용성과 유지보수성을 높이는 것을 권장합니다.

제안하는 리팩토링

공통 로직을 추출한 커스텀 훅 생성:

// src/pages/@owner/menu/hooks/use-preserved-form-data.ts
import { useLocation } from 'react-router-dom';
import type { FoodTruckFormData } from '@pages/@owner/food-truck-form/types';

interface LocationState {
  formData?: FoodTruckFormData;
}

export const usePreservedFormData = () => {
  const location = useLocation() as { state?: LocationState };
  return location.state?.formData;
};

그런 다음 각 mutation hook에서 사용:

+import { usePreservedFormData } from './use-preserved-form-data';
+
 export const useRegisterMenuMutation = (foodTruckId: number) => {
   const queryClient = useQueryClient();
   const navigate = useNavigate();
   const toast = useToast();
-  const location = useLocation();
-
-  const foodTruckFormData = location.state?.formData;
+  const foodTruckFormData = usePreservedFormData();

Also applies to: 72-74, 119-121

src/pages/@owner/menu/hooks/use-menu-edit.ts (1)

12-14: 타입 안전성 개선 필요 (이전 파일과 동일한 패턴)

use-menu-register.ts와 동일한 타입 안전성 및 변수명 문제가 있습니다. 세 파일 모두에서 동일한 패턴을 사용하므로, 공통 유틸리티로 추출하면 타입 안전성을 한 곳에서 관리할 수 있습니다.

앞서 제안한 usePreservedFormData 커스텀 훅을 이 파일에도 적용하세요.

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Learnt from: holdn2
Repo: CHA-CALL/ChaCall-FE PR: 214
File: src/pages/@owner/food-truck-form/utils/format-food-truck-form.ts:15-15
Timestamp: 2026-01-06T03:26:46.868Z
Learning: In the ChaCall-FE codebase, region.id in RegionResponse is marked as optional in the Swagger spec, but in practice it cannot be undefined when used in the food truck form context. Exception handling for invalid region data is already in place elsewhere in the codebase, so non-null assertions on region.id are acceptable in format-food-truck-form.ts.

Learnt from: holdn2
Repo: CHA-CALL/ChaCall-FE PR: 214
File: src/pages/@owner/food-truck-form/utils/format-food-truck-form.ts:15-15
Timestamp: 2026-01-06T03:26:46.868Z
Learning: In the ChaCall-FE codebase, region.id in RegionResponse is marked as optional in the Swagger spec, but in practice it cannot be undefined when used in the food truck form context. Exception handling for invalid region data is already in place elsewhere in the codebase, so non-null assertions on region.id are acceptable in format-food-truck-form.ts.

Learnt from: holdn2
Repo: CHA-CALL/ChaCall-FE PR: 214
File: src/pages/@owner/food-truck-form/utils/format-food-truck-form.ts:0-0
Timestamp: 2026-01-06T03:26:08.240Z
Learning: In the ChaCall-FE codebase, region.code in RegionResponse is marked as optional in the Swagger spec, but in practice it cannot be undefined when used in the food truck form context. Exception handling for invalid region data is already in place elsewhere in the codebase, so non-null assertions on region.code are acceptable in format-food-truck-form.ts.

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Learnt from: holdn2
Repo: CHA-CALL/ChaCall-FE PR: 214
File: src/pages/@owner/food-truck-form/utils/format-food-truck-form.ts:15-15
Timestamp: 2026-01-06T03:26:46.868Z
Learning: In the ChaCall-FE codebase, region.id in RegionResponse is marked as optional in the Swagger spec, but in practice it cannot be undefined when used in the food truck form context. Exception handling for invalid region data is already in place elsewhere in the codebase, so non-null assertions on region.id are acceptable in format-food-truck-form.ts.

src/pages/@owner/food-truck-form/schemas/food-truck-form.schema.ts (1)

20-21: 중복 확인 검증 로직을 개선하세요.

isNameDuplicated가 반드시 false여야 한다면, 명시적인 에러 메시지를 제공하는 것이 좋습니다.

♻️ 개선된 검증 로직

-  isNameChecked: z.boolean(),
-  isNameDuplicated: z.boolean().refine(v => v === false),
+  isNameChecked: z.boolean().refine(v => v === true, {
+    message: FOOD_TRUCK_ERROR_MESSAGE.nameDuplicate.required,
+  }),
+  isNameDuplicated: z.boolean().refine(v => v === false, {
+    message: FOOD_TRUCK_ERROR_MESSAGE.nameDuplicate.duplicated,
+  }),

src/pages/@owner/food-truck-form/hooks/use-food-truck-form.ts (1)

69-82: 데이터 로딩 useEffect의 의존성 배열을 확인하세요.

reset과 setValue를 의존성 배열에 포함하면 불필요한 재실행이 발생할 수 있습니다. React Hook Form의 이러한 함수들은 참조가 안정적이지만, 명시적으로 제외하는 것을 고려하세요.

♻️ 개선 제안

  useEffect(() => {
    if (!foodTruckDetailData) {
      setValue('menus', menus && menus.length > 0, { shouldValidate: true });
      return;
    }

    const result = resetFoodTruckFormValue(foodTruckDetailData, menus);

    if (result.isError) {
      toast.error('잘못된 정보입니다. 다시 시도해주세요.');
      return;
    }
    reset(result.values);
-  }, [foodTruckDetailData, menus, toast, reset, setValue]);
+  }, [foodTruckDetailData, menus, toast]);

또는 ESLint 경고를 방지하려면 주석으로 명시:

  }, [foodTruckDetailData, menus, toast, reset, setValue]); // reset과 setValue는 안정적인 참조

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Learnt from: holdn2
Repo: CHA-CALL/ChaCall-FE PR: 214
File: src/pages/@owner/menu/hooks/use-menu-list.ts:61-65
Timestamp: 2026-01-06T04:01:35.227Z
Learning: In src/pages/owner/menu/hooks/use-menu-list.ts, formData validation is only required when navigating to FOOD_TRUCK_FORM (via handleClickBack), not when navigating to MENU_REGISTER (via handleRegister), as MENU_REGISTER doesn't require formData to be defined.

Learnt from: holdn2
Repo: CHA-CALL/ChaCall-FE PR: 214
File: src/pages/@owner/food-truck-form/utils/format-food-truck-form.ts:15-15
Timestamp: 2026-01-06T03:26:46.868Z
Learning: In the ChaCall-FE codebase, region.id in RegionResponse is marked as optional in the Swagger spec, but in practice it cannot be undefined when used in the food truck form context. Exception handling for invalid region data is already in place elsewhere in the codebase, so non-null assertions on region.id are acceptable in format-food-truck-form.ts.

Learnt from: holdn2
Repo: CHA-CALL/ChaCall-FE PR: 214
File: src/pages/@owner/menu/hooks/use-menu-list.ts:61-65
Timestamp: 2026-01-06T04:01:35.227Z
Learning: In src/pages/owner/menu/hooks/use-menu-list.ts, formData validation is only required when navigating to FOOD_TRUCK_FORM (via handleClickBack), not when navigating to MENU_REGISTER (via handleRegister), as MENU_REGISTER doesn't require formData to be defined.

Learnt from: holdn2
Repo: CHA-CALL/ChaCall-FE PR: 214
File: src/pages/@owner/food-truck-form/utils/format-food-truck-form.ts:15-15
Timestamp: 2026-01-06T03:26:46.868Z
Learning: In the ChaCall-FE codebase, region.id in RegionResponse is marked as optional in the Swagger spec, but in practice it cannot be undefined when used in the food truck form context. Exception handling for invalid region data is already in place elsewhere in the codebase, so non-null assertions on region.id are acceptable in format-food-truck-form.ts.

Learnt from: holdn2
Repo: CHA-CALL/ChaCall-FE PR: 214
File: src/pages/@owner/food-truck-form/utils/format-food-truck-form.ts:0-0
Timestamp: 2026-01-06T03:26:08.240Z
Learning: In the ChaCall-FE codebase, region.code in RegionResponse is marked as optional in the Swagger spec, but in practice it cannot be undefined when used in the food truck form context. Exception handling for invalid region data is already in place elsewhere in the codebase, so non-null assertions on region.code are acceptable in format-food-truck-form.ts.

src/pages/@owner/food-truck-form/schemas/food-truck-form.schema.ts (1)

8-13: isNameDuplicated 검증에 에러 메시지 추가 권장

z.boolean().refine(v => v === false)는 함수 참조 가능하지만, 실패 시 generic 에러를 노출합니다. 더 명확한 UX를 위해 다음과 같이 개선할 수 있습니다:

isNameDuplicated: z.boolean().refine(
  v => v === false,
  { message: FOOD_TRUCK_ERROR_MESSAGE.name.duplicated }
)
// 또는 더 간단하게:
isNameDuplicated: z.literal(false)

참고: validateFoodTruckFormTime는 Zod v4.1.8에서 superRefine((value, ctx) => ...)의 정확한 시그니처를 구현하고 있으며, z.custom<RegionResponse>() / z.custom<AvailableDate>()는 런타임 검증 없이 타입만 제공하는 현재 패턴이 일관되게 적용되고 있습니다.

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Learnt from: holdn2
Repo: CHA-CALL/ChaCall-FE PR: 214
File: src/pages/@owner/menu/hooks/use-menu-list.ts:61-65
Timestamp: 2026-01-06T04:01:35.227Z
Learning: In src/pages/owner/menu/hooks/use-menu-list.ts, formData validation is only required when navigating to FOOD_TRUCK_FORM (via handleClickBack), not when navigating to MENU_REGISTER (via handleRegister), as MENU_REGISTER doesn't require formData to be defined.

Learnt from: holdn2
Repo: CHA-CALL/ChaCall-FE PR: 214
File: src/pages/@owner/food-truck-form/utils/format-food-truck-form.ts:15-15
Timestamp: 2026-01-06T03:26:46.868Z
Learning: In the ChaCall-FE codebase, region.id in RegionResponse is marked as optional in the Swagger spec, but in practice it cannot be undefined when used in the food truck form context. Exception handling for invalid region data is already in place elsewhere in the codebase, so non-null assertions on region.id are acceptable in format-food-truck-form.ts.

Learnt from: holdn2
Repo: CHA-CALL/ChaCall-FE PR: 214
File: src/pages/@owner/menu/hooks/use-menu-list.ts:61-65
Timestamp: 2026-01-06T04:01:35.227Z
Learning: In src/pages/owner/menu/hooks/use-menu-list.ts, formData validation is only required when navigating to FOOD_TRUCK_FORM (via handleClickBack), not when navigating to MENU_REGISTER (via handleRegister), as MENU_REGISTER doesn't require formData to be defined.

Learnt from: holdn2
Repo: CHA-CALL/ChaCall-FE PR: 214
File: src/pages/@owner/food-truck-form/utils/format-food-truck-form.ts:15-15
Timestamp: 2026-01-06T03:26:46.868Z
Learning: In the ChaCall-FE codebase, region.id in RegionResponse is marked as optional in the Swagger spec, but in practice it cannot be undefined when used in the food truck form context. Exception handling for invalid region data is already in place elsewhere in the codebase, so non-null assertions on region.id are acceptable in format-food-truck-form.ts.

Learnt from: holdn2
Repo: CHA-CALL/ChaCall-FE PR: 214
File: src/pages/@owner/food-truck-form/utils/format-food-truck-form.ts:0-0
Timestamp: 2026-01-06T03:26:08.240Z
Learning: In the ChaCall-FE codebase, region.code in RegionResponse is marked as optional in the Swagger spec, but in practice it cannot be undefined when used in the food truck form context. Exception handling for invalid region data is already in place elsewhere in the codebase, so non-null assertions on region.code are acceptable in format-food-truck-form.ts.